Skip to content

feat(retain): add retain method - #366

Merged
Kerollmops merged 1 commit into
RoaringBitmap:mainfrom
HellButcher:feat/retain
Oct 26, 2026
Merged

Kerollmops merged 1 commit into
RoaringBitmap:mainfrom
HellButcher:feat/retain

Conversation

@HellButcher

Copy link
Copy Markdown
Contributor

adds a retain(&mut self, f: FnMut(u32) -> bool) method to RoaringBitmap and RoaringTreemap.
It works similar to Vec::retain.

@HellButcher
HellButcher force-pushed the feat/retain branch 2 times, most recently from 31a2549 to cc2f119 Compare September 15, 2026 10:41

@Kerollmops Kerollmops left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @HellButcher 👋

Thank you for the nice addition. Would you mind fixing the CI errors, please?

@Kerollmops Kerollmops left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Final little clippy issue now 👀 😊

adds a `retain(&mut self, f: FnMut(u32) -> bool)` method to
`RoaringBitmap` and `RoaringTreemap`.
It works similar to `Vec::retain`.

@Kerollmops Kerollmops left a comment •

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the feature and PR fixes, merging 👍

@Kerollmops
Kerollmops added this pull request to the merge queue Oct 26, 2026
Merged via the queue into RoaringBitmap:main with commit a2f6374 Oct 26, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants